Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rolls clang to 725656bdd885483c39f482a01ea25d67acf39c46 #49248

Merged
merged 1 commit into from
Dec 19, 2023

Conversation

zanderso
Copy link
Member

@zanderso zanderso commented Dec 19, 2023

We're on a version of clang that crashes a lot on macOS. This PR rolls clang forward to a new release from the Fuchsia toolchain team. If it does not fix the crash, we can roll back with #49249. Both rolling forward and back require the fixes to the .mm file in this PR.

Copy link
Contributor

@flar flar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just to clarify (chatted with Zach offline about this), but the new asserts are required to prove to clang-tidy that the values can't be null otherwise it thinks we have a problem there.

LGTM

@zanderso zanderso merged commit df1593e into flutter:main Dec 19, 2023
28 checks passed
@zanderso zanderso deleted the roll-clang branch December 19, 2023 21:35
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Dec 19, 2023
auto-submit bot pushed a commit to flutter/flutter that referenced this pull request Dec 19, 2023
…140422)

flutter/engine@3b156c8...df1593e

2023-12-19 zanderso@users.noreply.github.com Rolls clang to 725656bdd885483c39f482a01ea25d67acf39c46 (flutter/engine#49248)
2023-12-19 aam@google.com Roll dart to 3.3.0-242.0.dev (flutter/engine#49239)

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC jimgraham@google.com,rmistry@google.com,zra@google.com on the revert to ensure that a human
is aware of the problem.

To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
CoderDake pushed a commit to CoderDake/flutter that referenced this pull request Dec 28, 2023
…lutter#140422)

flutter/engine@3b156c8...df1593e

2023-12-19 zanderso@users.noreply.github.com Rolls clang to 725656bdd885483c39f482a01ea25d67acf39c46 (flutter/engine#49248)
2023-12-19 aam@google.com Roll dart to 3.3.0-242.0.dev (flutter/engine#49239)

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC jimgraham@google.com,rmistry@google.com,zra@google.com on the revert to ensure that a human
is aware of the problem.

To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants